Skip to content

Publish SNAPSHOT artifacts during PR builder#875

Closed
thibauult wants to merge 2 commits into
finos:mainfrom
thibauult:cicd/publish-snapshots-in-pr-builder
Closed

Publish SNAPSHOT artifacts during PR builder#875
thibauult wants to merge 2 commits into
finos:mainfrom
thibauult:cicd/publish-snapshots-in-pr-builder

Conversation

@thibauult

@thibauult thibauult commented Jun 10, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds a Publish SNAPSHOT artifacts step to the PR builder workflow
  • The step only runs on pull_request events (skipped on pushes to main/*-rc)
  • Uses publishToSonatype which automatically targets the Sonatype snapshot repository for -SNAPSHOT versions
  • No GPG signing required — already gated on isReleaseVersion in bdk.java-publish-conventions.gradle

Prerequisites

MAVEN_USERNAME and MAVEN_PASSWORD repository secrets must be set (already used by release.yml).

Test plan

  • Open a PR and verify the Publish SNAPSHOT artifacts step runs and succeeds
  • Verify artifacts are published to https://ossrh-staging-api.central.sonatype.com/content/repositories/snapshots/
  • Verify the step is skipped on a direct push to main

@linux-foundation-easycla

linux-foundation-easycla Bot commented Jun 10, 2026

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: thibauult / name: Thibault Pensec (2331cbf)

@thibauult thibauult force-pushed the cicd/publish-snapshots-in-pr-builder branch from d301b60 to 2331cbf Compare June 10, 2026 08:11
…tion@v4

The old gradle/wrapper-validation-action is deprecated and was causing CI failures.
@thibauult

Copy link
Copy Markdown
Member Author

Closing in favour of a new PR from the upstream branch so that repository secrets are available to the CI workflow.

@thibauult thibauult closed this Jun 10, 2026
@thibauult thibauult deleted the cicd/publish-snapshots-in-pr-builder branch June 10, 2026 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant